Extract clean_benches function

Extract clean_tests function

Extract clean_examples function

Extract clean_bins function

Extract clean_library function

Don't store root in the layout

Unify naming for `package_root`

Superfluous pub

Restructure code

Make layout private to targets

Move code for inferring build targets to a separate file

Fix typo

Refactor: virtual manifest does not need layout

Simplify

Simplify

Make `Layout` a private implementation detail of toml

Auto merge of #4257 - alexcrichton:update, r=alexcrichton

Update dependencies with semver-compat version

Just a standard update

Update dependencies with semver-compat version

Just a standard update

Auto merge of #4248 - dethoter:issue-4228, r=alexcrichton

bench command supports --no-fail-fast flag.

Fix for #4228.

Auto merge of #4249 - tmccombs:zsh-test-update, r=alexcrichton

Add more options for cargo test zsh completion

Add more options for cargo test zsh completion

bench command supports --no-fail-fast flag.

Auto merge of #4246 - est31:master, r=Mark-Simulacrum

Let two further tests tolerate col info in panics

Needed by https://github.com/rust-lang/rust/pull/42938

I've now ripgrepped for "panicked at" and found no further
test that hardcodes the "filename:line$" format.

Let two further tests tolerate col info in panics

Needed by https://github.com/rust-lang/rust/pull/42938

I've now ripgrepped for "panicked at" and found no further
test that hardcodes the "filename:line$" format.

Auto merge of #4244 - est31:master, r=matklad

Let the cargo_bench_failing_test tolerate col info

Needed by https://github.com/rust-lang/rust/pull/42938

Let the cargo_bench_failing_test tolerate col info

Needed by https://github.com/rust-lang/rust/pull/42938

Auto merge of #4239 - Keruspe:master, r=alexcrichton

hash '__CARGO_DEFAULT_LIB_METADATA' in metadata for rustc

We already have __CARGO_DEFAULT_LIB_METADATA to force adding the hash of some metadata in the libraries file name when building rust.
For now, we only check if it is set or not.
This patch makes also use of its value to compute the hash. This way we'll be able to pass in the channel we're building to avoid hash collisions between channels.

Auto merge of #4214 - msehnout:master, r=matklad

Infer multi-file binaries like `src/bin/server/main.rs` by convention

This feature is described in issue #4086

Auto merge of #4238 - ashleygwilliams:policies, r=steveklabnik

updates to policies

based on convo with @withoutboats and @carols10cents

hash '__CARGO_DEFAULT_LIB_METADATA' in metadata for rustc

This will let us seed the release channel as part of the
metadata used for the hash to avoid conflicting hashes across
channels.

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

Add documentation for multi-file binary convention

Prevent unnecessary heap allocations and simplify condition

Auto merge of #4234 - michaelwoerister:smarter-cargo-incremental-2, r=alexcrichton

Set -Ccodegen-units=N for non-path dependencies when CARGO_INCREMENTAL is set.

This PR tweaks compilation for non-path dependencies when CARGO_INCREMENTAL is set. Before, `cargo` would compile such dependencies "just" non-incrementally while, with this PR, it will also set the number of CGUs to the number of CPU cores. Otherwise compiling those dependencies might take significantly longer (because of more expensive optimizations and less multi-threading) than compiling them incrementally -- which is what we wanted to avoid in the first place.

feat(policy): add information on security disclosures

fix(doc): clarify the content the code of conduct covers

"main" can be valid name for binary again, improve test

Auto merge of #4229 - azerupi:install-version-range, r=matklad

Implement semver ranges for install --vers

This implements the design discussed in #4207
It allows to specify semver ranges on `cargo install ... --vers`

1. The first character of the `--vers` value is checked, if there is none we return an error.
2. If it is one of `<`, `>`, `=`, `^`, `~` we parse the value as a `VersionReq`, otherwise we parse it as a `Version`.
3. If the parsing as a `Version` fails but parsing as `VersionReq` succeeds, we add a note to the warning mentioning that a qualifier should be used to specify a semver range. This catches versions with less than tree digits.

Otherwise, the previous behaviour is preserved with the warning of backwards compatibility.

This means that

- `cargo install ... --vers "^1.2.3"` will be parsed as a range
- `cargo install ... --vers 1.2.3` will be parsed as a version
- `cargo install ... --vers 1.2` will be parsed as a version for backwards compatibility reasons, fail and be passed through as is,**but** we will add a note `if you want to specify semver range, add an explicit qualifier, like ^1.2`
- `cargo install ... --vers blah` will be parsed as a version for backwards compatibility reasons (which is weird because it is not even a valid semver range) and produce an `unknown error` down the line. I have left this behaviour untouched because it worked like that before, but I can easily make it error sooner with a better message.

Set -Ccodegen-units=N for non-path dependencies when CARGO_INCREMENTAL is set.

Infer paths as well as names for binaries

Remove extra blank lines and commas on last match arm for consistency

Auto merge of #4230 - Mark-Simulacrum:double-print, r=alexcrichton

Prevent rustc stderr/stdout from being duplicated.

Please review carefully. I've not submitted patches to Cargo before, I think, so this may be flawed in some way I haven't detected yet. Tests are green locally, though.

Fixes #4223

Prevent rustc stderr/stdout from being duplicated.

Auto merge of #4227 - YaLTeR:version-field-fix, r=alexcrichton

Clarify how breaking changes should be handled before 1.0.0

Resolves #4064

Implement semver ranges for install --vers

Clarify how breaking changes should be handled before 1.0.0

Auto merge of #4218 - alexcrichton:fix-flaky, r=alexcrichton

Ignore a test that accesses crates.io for now

Tests in general shouldn't hit the network, but this one requires a crate from
crates.io :(

Ignore a test that accesses crates.io for now

Tests in general shouldn't hit the network, but this one requires a crate from
crates.io :(

Auto merge of #4209 - natboehm:add-index-flag, r=alexcrichton

Replace `cargo publish --host` with `cargo publish --index`

This change makes the command consistent with other versions with the same flag. `cargo publish --host` is still supported, currently marked as deprecated, or for reuse as mentioned in issue #4208.

Fixes #3797

Auto merge of #4215 - alexcrichton:search-stdout, r=matklad

Switch `cargo search` output to stdout

Closes #4203

Switch `cargo search` output to stdout

Closes #4203

Infer multi-file binaries like `src/bin/server/main.rs` by convention

Added comments indicating which parts should be removed at appropriate time

Add test for if user ever decides to use both --index and --host

replace instances of flag --host with --index in publish, still supports --host however deprecation warning is issued

Auto merge of #4201 - alexcrichton:more-human, r=matklad

Flag crate registry errors as "human"

They often contain useful information!

Closes #4200

Auto merge of #4202 - dethoter:issue-4199, r=alexcrichton

Fix an incorrect merge of credentials. Add a new test for that.

Fix for #4199.

Fix an incorrect merge of credentials. Add a new test for that.

Flag crate registry errors as "human"

They often contain useful information!

Closes #4200

Auto merge of #4196 - gilescope:multiline_assert, r=alexcrichton

Multiline assert

Found a second location where multiline assert_eq failure breaks a test.
This is an extention of #4181.

Auto merge of #4180 - pnkfelix:issue-4135, r=alexcrichton

Issue 4135: include = [...] should override git file list

Fix #4135: if theres `include = [...]`, then do not prepopulate file list via git.

Merge branch 'master' into multiline_assert

Regression test for issue #4135.

Prep for asser_eq failure msg format change.

Auto merge of #4190 - alexcrichton:wincosole-fix, r=alexcrichton

Update wincolor

Helps mitigate #4189

Update wincolor

Helps mitigate #4189

Auto merge of #4181 - gilescope:patch-1, r=alexcrichton

Make test less brittle prior to assert_eq failure message format change

PR required for rust-lang/rust#42541 to make assert_eq error message be multi-line. Before implementing this we need to make the current test less brittle.
Not 100% clear on if I need the final [...] or not.

Fixing test failure

Auto merge of #4182 - matklad:simplify, r=alexcrichton

Simplify inferred binary names

Fix for https://github.com/rust-lang/rust/issues/42365, which probably breaks other stuff :(

Auto merge of #4185 - ids1024:typo, r=Mark-Simulacrum

Fix typo in comment

Fix typo in comment

Fix test on windows

Backslashes here are escaped (\\), so [/] does not really help :(

Auto merge of #4178 - nrc:shell, r=alexcrichton

Support having a generic writer instead of a termcolor StandardStream

This is requied by the RLS (and presumably any other client who wants to use Cargo as a lib and redirect output)

r? @alexcrichton

Support having a generic writer instead of a termcolor StandardStream

This is requied by the RLS (and presumably any other client who wants to use Cargo as a lib and redirect output)

Don't search bins in src if there's a library

Ideally we want not to search for binaries there in any case, but that's
probably not gonna happen due to backwards comparability.

Simplify inferred binary names

Make test less brittle prior to assert_eq failure message format change

PR #42541 on Rust is to make assert_eq error message be multi-line. Before implementing this we need to make the current test less brittle.

Fix #4135: if theres `include = [...]`, then dont prepopulate filelist via git

Factor search for sibling Cargo.toml and .git folder into separate subroutine.

assert_eq failure message now multi-line related to rust pull request 42541.

Auto merge of #4125 - alexcrichton:rerun-if-env-changed, r=matklad

Implement `cargo:rerun-if-env-changed=FOO`

This commit implements a new method of rerunning a build script if an
environment variable changes. Environment variables are one of the primary
methods of giving inputs to a build script today, and this'll help situations
where if you change an env var you don't have to remember to clean out an old
build directory to ensure fresh results.

Closes #2776

Auto merge of #4176 - alexcrichton:atty, r=matklad

Use the `atty` crate for TTY detection

This is more robust in the face of msys terminals and otherwise helps share more
dependencies!

Closes #4166

Use the `atty` crate for TTY detection

This is more robust in the face of msys terminals and otherwise helps share more
dependencies!

Closes #4166

Implement `cargo:rerun-if-env-changed=FOO`

This commit implements a new method of rerunning a build script if an
environment variable changes. Environment variables are one of the primary
methods of giving inputs to a build script today, and this'll help situations
where if you change an env var you don't have to remember to clean out an old
build directory to ensure fresh results.

Closes #2776

Auto merge of #4167 - alexcrichton:keep-metadata, r=matklad

Preserve `package.metadata` when packaging

Now that we use Serde this is actually trivial to implement!

Closes #4142

Auto merge of #4168 - alexcrichton:more-metadata, r=alexcrichton

Hash `Kind` in metadata for crates

This fixes what is now a bug in Cargo discovered when attempting to land
rust-lang/rust#42495 where Cargo will compile both host and target artifacts
with the same `-C metadata` flag. This means that the compile can load two
crates with the same SVH that are supposed to be distinct, which causes weird
bugs with crate loading and whatnot.

This commit throws in the `Kind` into the metadata calculation to resolve this
to ensure that host/target artifacts always have a different `-C metadata`

Hash `Kind` in metadata for crates

This fixes what is now a bug in Cargo discovered when attempting to land
rust-lang/rust#42495 where Cargo will compile both host and target artifacts
with the same `-C metadata` flag. This means that the compile can load two
crates with the same SVH that are supposed to be distinct, which causes weird
bugs with crate loading and whatnot.

This commit throws in the `Kind` into the metadata calculation to resolve this
to ensure that host/target artifacts always have a different `-C metadata`

Preserve `package.metadata` when packaging

Now that we use Serde this is actually trivial to implement!

Closes #4142

Auto merge of #4162 - alexcrichton:termcolor, r=matklad

Migrate from the `term` crate to `termcolor`

The API of `termcolor` fits what the system gives us much more nicely and should
be well battle-tested from ripgrep. Additionally we don't really need huge
terminfo parsers, that wasn't every really the intention of the color support
here.

Auto merge of #4155 - ketralnis:fossil, r=alexcrichton

Add fossil VCS support to `cargo new`

Fossil is a simple, high-reliability, distributed software configuration management system <https://www.fossil-scm.org/>

I mostly followed https://github.com/rust-lang/cargo/pull/3842 as a template. Like that one, this only adds support for `cargo new`, not for pulling down fossil-hosted dependencies

A problem that i didn't tackle but I'd be willing to is a little more more `trait`ifying of the VCSs. I would need some guidance on that since it looks like git has some more thorough support than e.g. hg does but it looks pretty doable

Auto merge of #4159 - matklad:arch, r=alexcrichton

Blurb about Cargo inner workings

Hi!

At today's dev tools meetings we've discussed how we can foster contributions to rust dev tools, and I've complained that some tools are difficult to contribute to because it's difficult to learn how they work because there are no docs, and some other people complained that Cargo is difficult to contribute to because it is rather complex. And, as a member of Cargo team, I thought that I am probably responsible for fixing that :)

So, here's my take at making it easier to dive in into Cargo! I've written a small birds eye overview of the current architecture of Cargo (more like a list of things to look at while reading the code actually :)

In general, I am skeptical about documenting internals of binaries (docs will become obsolete, and very fast), but such a high level picture should be pretty robust (I deliberately avoided linking to the actual source code), and so pretty low-effort to maintain. We do something similar for IntelliJ Rust: https://github.com/intellij-rust/intellij-rust/blob/master/ARCHITECTURE.md as well.

r? @alexcrichton

Auto merge of #4158 - stepancheg:hg-ignore, r=alexcrichton

Generate .hgignore with syntax:glob

Fixes #4005

Migrate from the `term` crate to `termcolor`

The API of `termcolor` fits what the system gives us much more nicely and should
be well battle-tested from ripgrep. Additionally we don't really need huge
terminfo parsers, that wasn't every really the intention of the color support
here.

Auto merge of #4149 - ordian:master, r=alexcrichton

Drop rustc-serialize dependency

Fixes #3944.

Auto merge of #3978 - dethoter:separated-credentials, r=alexcrichton

Move API token into the separate file.

Fix of #3748. BTW, it's not clear what to do with old config.
Should I add a check for old config and try to remove [repository.token] field from it every time user add a new token?
Or should I just prefer to use a token field from a new config over the old one?

Fix typos and reword slightly

Auto merge of #4163 - alexcrichton:update, r=alexcrichton

Update dependencies

Brings in a few fixes for Windows-related dependencies

Update dependencies

Brings in a few fixes for Windows-related dependencies

Blurb about Cargo inner workings

Generate .hgignore with syntax:glob

Fixes #4005

Add fossil VCS support to `cargo new`

Fossil is a simple, high-reliability, distributed software configuration
management system <https://www.fossil-scm.org/>